feat : 지명 검색결과 없는 경우 추가#25
Hidden character warning
Conversation
WalkthroughLocationBottomSheet의 결과 영역이 전체 너비 Box로 감싸져 레이아웃(weight, padding, 배경)이 변경되고, 결과가 있을 때는 LazyColumn으로 항목 리스트를 렌더링하며, 결과가 없고 쿼리가 비어있지 않으면 중앙에 "검색 결과가 없습니다." 메시지를 표시하도록 조건부 렌더링 흐름이 재구성되었습니다. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧹 Recent nitpick comments
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🔇 Additional comments (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In
@core/designsystem/src/main/java/com/umcspot/spot/designsystem/component/bottomsheet/LocationBottomSheet.kt:
- Around line 271-286: The bottom navigation padding navBarPadding is applied
twice (to the outer Box and again to the LazyColumn), causing excess bottom
spacing; remove the duplicate padding from the LazyColumn by deleting the
.padding(bottom = navBarPadding) call in the LazyColumn modifier and rely on the
outer Box's .padding(bottom = navBarPadding) to provide the correct inset so
results and selection logic (selected, results) render with the intended bottom
spacing.
🧹 Nitpick comments (2)
core/designsystem/src/main/java/com/umcspot/spot/designsystem/component/bottomsheet/LocationBottomSheet.kt (2)
278-287: 상단HorizontalDivider가 가려질 수 있습니다.
Box내에서HorizontalDivider와LazyColumn이 형제 요소로 배치되어 있습니다.Box는 자식을 겹쳐서 렌더링하므로, 흰색 배경을 가진LazyColumn이 상단 구분선을 덮을 수 있습니다.
Column으로 감싸거나 구분선을LazyColumn내부로 이동하는 것을 고려해 주세요.♻️ Column 사용 예시
Box( modifier = Modifier .fillMaxWidth() .weight(1f) .padding(bottom = navBarPadding) .background(SpotTheme.colors.white) ) { if (results.isNotEmpty()) { val isMaxSelected = selected.size >= 10 - HorizontalDivider(thickness = 0.5.dp, color = SpotTheme.colors.G200) - LazyColumn( - modifier = Modifier - .fillMaxWidth() - .background(SpotTheme.colors.white), - ) { + Column { + HorizontalDivider(thickness = 0.5.dp, color = SpotTheme.colors.G200) + LazyColumn( + modifier = Modifier + .fillMaxWidth() + .weight(1f) + .background(SpotTheme.colors.white), + ) {
308-317: 이미 선택된 항목 클릭 시 동작 확인 필요.이미 선택된 항목(
isAlreadySelected = true)의 경우 클릭은 활성화되어 있지만(enabled = true), 실제로 아무 동작도 수행되지 않습니다. 사용자가 클릭해도 피드백이 없어 혼란스러울 수 있습니다.의도된 동작이라면
enabled = !isMaxSelected && !isAlreadySelected로 변경하여 비활성화 처리하거나, 클릭 시 선택 해제(onRemoveSelected) 동작을 추가하는 것을 고려해 주세요.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
core/designsystem/src/main/java/com/umcspot/spot/designsystem/component/bottomsheet/LocationBottomSheet.kt
🔇 Additional comments (1)
core/designsystem/src/main/java/com/umcspot/spot/designsystem/component/bottomsheet/LocationBottomSheet.kt (1)
326-335: 검색 결과 없음 처리가 잘 구현되었습니다.
query.isNotBlank()조건을 통해 실제 검색이 수행된 경우에만 메시지를 표시하고, 초기 상태에서는 빈 화면을 유지합니다. PR 목표에 부합하는 적절한 구현입니다.
...tem/src/main/java/com/umcspot/spot/designsystem/component/bottomsheet/LocationBottomSheet.kt
Show resolved
Hide resolved
1 participant
Related issue 🛠
Work Description 📝
Screenshot 📸
Uncompleted Tasks 😅
PR Point 📌
트러블 슈팅 💥
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.